Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade AppAuth-iOS to 1.0.0 #338

Merged
merged 6 commits into from
Jun 28, 2019
Merged

Conversation

kadikraman
Copy link
Contributor

@kadikraman kadikraman commented Jun 27, 2019

Fixes #337
Fixes #339

Description

Upgrades the native AppAuth-iOS dependency.

Note: this will have to be a major release, due to a typo fix in a method name

@kadikraman kadikraman added this to the Next Major Release 🚀 milestone Jun 27, 2019
Copy link

@thymikee thymikee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for taking care if it works with autolinking :)

README.md Outdated

Then run `pod install`. Note that version 0.94 is the first of the library to support iOS 11.
Then run `pod install`.
Copy link

@thymikee thymikee Jun 28, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, all instructions about linking on both paltforms are obsolete in RN 0.60 if the library works with autolinking. All you need to document now is:

  1. running pod install after adding the dep
  2. how to use with Carthage or anything that is not CocoaPods or manual link

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, I'll update the instructions. Thanks for all your help on this!

@kadikraman
Copy link
Contributor Author

Note - next major release will have RN0.60 as a peer dependency, so the instructions for <RN0.60 will be removed before then. I'm leaving both instructions in (for now), for users who are installing the library between now and the next major release, and are using the instructions from the master branch on GitHub.

@kadikraman kadikraman merged commit f160144 into master Jun 28, 2019
@kadikraman kadikraman deleted the feature/upgrade-appauth-ios branch June 28, 2019 19:30
@kadikraman kadikraman restored the feature/upgrade-appauth-ios branch July 4, 2019 16:00
@kadikraman
Copy link
Contributor Author

Released in v5.0.0 🎉

@kadikraman kadikraman deleted the feature/upgrade-appauth-ios branch January 10, 2020 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add new pod installation instructions Update to use AppAuth 1.0.0
2 participants